Auto merge of #3350 - alexcrichton:more-tweaks, r=alexcrichton

Slight tweaks to CI

* Pass `--quiet` to all tests to have some quieter output
* Skip builds on the `master` branch as `auto` is already checked
* Check the right env var for repo branches on appveyor

Slight tweaks to CI

* Pass `--quiet` to all tests to have some quieter output
* Skip builds on branches other than `auto-cargo` as it's already checked
* Check the right env var for repo branches on appveyor
* Only run a few builds on PRs

Auto merge of #3347 - ibabushkin:master, r=alexcrichton

Reordered rustc arguments.

This adresses #3276 .

Some tests I can't run locally might still fail, I'm fixing those.

The core change is rather trivial, simply reorder some existing code, and update tests to match the new results.

Fixed a few additional tests.

Fixed type breaking some tests.

Changed another batch of tests to account for new rustc argument order.

Auto merge of #3345 - alexcrichton:release-branches, r=alexcrichton

Add support for release branches in Cargo

Follow the same strategy as the compiler for now in basically every respect:

* Add new `--release-channel` configure option, defaulting to `dev`
* Remove old `--enable-nightly`
* Add `--enable-build-openssl` as an orthogonal option
* Hook up Travis/AppVeyor to stable/beta/master branches to do the right
  channel builds.

Add support for release branches in Cargo

Follow the same strategy as the compiler for now in basically every respect:

* Add new `--release-channel` configure option, defaulting to `dev`
* Remove old `--enable-nightly`
* Add `--enable-build-openssl` as an orthogonal option
* Hook up Travis/AppVeyor to stable/beta/master branches to do the right
  channel builds.

Another test update for the argument order changetest update for the argument order change. Please enter the commit message for your changes. Lines starting

Updated one more test to fit new rustc argument order.

[WIP] Reordered rustc arguments.

Auto merge of #3342 - alexcrichton:fix-ssl, r=alexcrichton

Bump git2 dep to fix SSL paths

This commit includes alexcrichton/git2-rs@a8f4a7faa which switches the order of
initialization of libgit2. That commit ensures that the relevant env vars which
a statically linked OpenSSL needs to function are set before libgit2 is
initialized to ensure that libgit2 uses them.

This was regressed accidentally in alexcrichton/git2-rs@071902aa when
initialization was tweaked.

Closes #3340

Bump git2 dep to fix SSL paths

This commit includes alexcrichton/git2-rs@a8f4a7faa which switches the order of
initialization of libgit2. That commit ensures that the relevant env vars which
a statically linked OpenSSL needs to function are set before libgit2 is
initialized to ensure that libgit2 uses them.

This was regressed accidentally in alexcrichton/git2-rs@071902aa when
initialization was tweaked.

Closes #3340

Auto merge of #3339 - SimonSapin:git-short-id, r=alexcrichton

Use short IDs in git dependencies checkout path

… in order to contribute less to the path length limit on Windows: https://github.com/servo/servo/pull/14397

Use short IDs in git dependencies checkout path

… in order to contribute less to the path length limit on Windows:
https://github.com/servo/servo/pull/14397

Auto merge of #3332 - alexcrichton:build-openssl, r=alexcrichton

Compile OpenSSL from source on OSX

I'm seeing a bunch of weird illegal instructions on OSX nightlies for
Cargo. My guess is that they're all related to OpenSSL linking. Right
now we're linking from Homebrew but I have a sneaking suspicion that it
compiles with `-march=native` rather than what we'd like as a portable
binary. To work around this compile OpenSSL ourselves and link it that
way.

Note that I believe this won't bring in the certificate trust store of
OpenSSL on OSX (or at least not the right one from the keychain). We
shouldn't need that, however, as OpenSSL is just used as the
cryptographic primitives in libssh2 and Cargo itself. So in that sense
we shouldn't need it for actually SSL at all.

Compile OpenSSL from source on OSX

I'm seeing a bunch of weird illegal instructions on OSX nightlies for
Cargo. My guess is that they're all related to OpenSSL linking. Right
now we're linking from Homebrew but I have a sneaking suspicion that it
compiles with `-march=native` rather than what we'd like as a portable
binary. To work around this compile OpenSSL ourselves and link it that
way.

Note that I believe this won't bring in the certificate trust store of
OpenSSL on OSX (or at least not the right one from the keychain). We
shouldn't need that, however, as OpenSSL is just used as the
cryptographic primitives in libssh2 and Cargo itself. So in that sense
we shouldn't need it for actually SSL at all.

Auto merge of #3331 - alexcrichton:fix-segfaults, r=alexcrichton

Update git2 fixing initialization races

This updates libgit2/libssh2 bindings to fix initialization races in
OpenSSL. This should fix some of the spurious segfaults we've been
seeing on Travis OSX.

Update git2 fixing initialization races

This updates libgit2/libssh2 bindings to fix initialization races in
OpenSSL. This should fix some of the spurious segfaults we've been
seeing on Travis OSX.

Auto merge of #3326 - tbu-:pr_bump_git2_061, r=alexcrichton

Bump git2 dependency to fix #3312

Bump git2 dependency to fix #3312

Auto merge of #3325 - alexcrichton:fix-flaky, r=alexcrichton

Make a freshness test less flaky

We can't rely on frobbing mtimes, need to actually just wait a whole
second.

Make a freshness test less flaky

We can't rely on frobbing mtimes, need to actually just wait a whole
second.

Auto merge of #3315 - alexcrichton:fix-travis, r=alexcrichton

Really fix OSX nightlies

After #3311 we're now correctly trying to link OpenSSL statically on
OSX. Unfortunately though this is failing to complete on the builders.
Turns out the way we install OpenSSL through Homebrew create "universal
archives" which is essentially an archive with both i686 and x86_64
object files, but separated. The linker takes care of this just fine but
rustc currently chokes on it, unable to include the library statically
into the compiler.

To work around this we prepare our own mini install of OpenSSL by
copying relevant bits into a local directory (like we do on Linux). As
part of this we use the `lipo` tool, which is used to manage these fat
archives, to disassemble the archive and only extract the relevant
architecture. This should make a pre-installation step which both
extracts the information and configures Cargo to use it.

This should also fix the errors we're seeing on Travis I believe.

Really fix OSX nightlies

After #3311 we're now correctly trying to link OpenSSL statically on
OSX. Unfortunately though this is failing to complete on the builders.
Turns out the way we install OpenSSL through Homebrew create "universal
archives" which is essentially an archive with both i686 and x86_64
object files, but separated. The linker takes care of this just fine but
rustc currently chokes on it, unable to include the library statically
into the compiler.

To work around this we prepare our own mini install of OpenSSL by
copying relevant bits into a local directory (like we do on Linux). As
part of this we use the `lipo` tool, which is used to manage these fat
archives, to disassemble the archive and only extract the relevant
architecture. This should make a pre-installation step which both
extracts the information and configures Cargo to use it.

This should also fix the errors we're seeing on Travis I believe.

Auto merge of #3311 - alexcrichton:fix-nightlies, r=brson

Link OpenSSL statically on OSX

Now that Cargo requires OpenSSL >= 1.0.1 transitively through the `openssl-sys`
0.9 release the dynamic libraries for OpenSSL are no longer located on OSX by
default. This means that the support necessary for libssh2 needs to be
statically linked rather than dynamically linked.

Closes #3303

Link OpenSSL statically on OSX

Now that Cargo requires OpenSSL >= 1.0.1 transitively through the `openssl-sys`
0.9 release the dynamic libraries for OpenSSL are no longer located on OSX by
default. This means that the support necessary for libssh2 needs to be
statically linked rather than dynamically linked.

Closes #3303

Auto merge of #3251 - alexcrichton:fix-regr, r=brson

Fix regression with path overrides

If an override points to a path dependency then that's not locked, so we're
missing information for that, but it's already warned about, so no need to
worry.

Closes #3288

Auto merge of #3299 - alexcrichton:fix-tests, r=alexcrichton

Fix tests on nightly

Fix tests on nightly

Auto merge of #3297 - alexcrichton:bump, r=brson

Bump to 0.16.0

Bump to 0.16.0

Auto merge of #3102 - nipunn1313:attempt, r=alexcrichton

Mix feature flags into fingerprint/metadata shorthash

Since building dependencies results in different libraries
depending on the feature flags, I added the feature flags into
the short_hash.

This solves an issue when multiple crates share a target directory
or multiple targets share a common library with divergent feature
flag choice.

I'm not sure if this architecturally the best way to solve this problem, but I did confirm that this fixes the issue I was seeing. I can also add a test for this case if this code is taking the right approach (or if it would help illustrate the issue).

Target_env -> Target_os

Ignore killing_cargo_releases_the_lock on windows

Merge branch 'master' into attempt

- Fix instances of try! converting to ? syntax

Fix regression with path overrides

If an override points to a path dependency then that's not locked, so we're
missing information for that, but it's already warned about, so no need to
worry.

Auto merge of #3286 - rjgoldsborough:build-script-docs-mistake-336, r=alexcrichton

fixing build script docs mistakes

Fixes https://github.com/rust-lang/crates.io/issues/336

Ignore the flaky lock test on older windows

Auto merge of #3283 - saschagrunert:master, r=alexcrichton

Changed try! macros to ? operator

Since the stabilization of the ? operator (release 1.13.0)
the ? operator should be used to use idiomatic Rust.

Auto merge of #3294 - sanxiyn:unused-type-alias, r=alexcrichton

Remove unused type aliases

Found by rust-lang/rust#37631 and necessary to land because of cargotest.

Remove unused type aliases

Changed rustversion to the last successful nightly

Auto merge of #3272 - cuviper:openssl-1.1.0, r=alexcrichton

Update dependencies for OpenSSL 1.1.0 compatibility

The primary targets here are openssl and openssl-sys crates 0.9,
bringing support for OpenSSL 1.1.0.  This requires updating the curl
and git2 related dependencies as well.

A small change is required in cargo itself for the new Hasher API.
Results from the hasher are simply unwrapped for now, matching the
Windows behavior that already panics on error.

More updates for OpenSSL 1.1.0

Changed rustversion to 2016-11-05

Fix one more [/] in env-var test for windows

Fix dll prefix/suffix test for windows

Fix [/] for windows on env_metadata test

Added back __CARGO_DEFAULT_LIB_METADATA handling with comment + test

fixing build script docs mistakes

Fixed too long lines

Changed rustversion to latest nightly release

Link targets in the FRESH step as well

Set OPENSSL_DIR for CI

Update dependencies for OpenSSL 1.1.0 compatibility

The primary targets here are openssl and openssl-sys crates 0.9,
bringing support for OpenSSL 1.1.0.  This requires updating the curl
and git2 related dependencies as well.

A small change is required in cargo itself for the new Hasher API.
Results from the hasher are simply unwrapped for now, matching the
Windows behavior that already panics on error.

Updated indent and rustversion.txt

Auto merge of #3284 - alexcrichton:fix-uploads, r=alexcrichton

Publish builds from the auto-cargo branch

Cargo doesn't use auto, it uses auto-cargo

Publish builds from the auto-cargo branch

Cargo doesn't use auto, it uses auto-cargo

Changed try! macros to ? operator

Since the stabilization of the ? operator (release 1.13.0)
the ? operator should be used to use idiomatic Rust.

Auto merge of #3280 - alexcrichton:test-features, r=brson

Fix passing --features when testing multiple packages

The wrong method was being passed to resolution accidentally. Features specified
via `--features` and `--no-default-features` are spec'd as only applying to the
*current* package, not all packages.

Closes #3279

Fix passing --features when testing multiple packages

The wrong method was being passed to resolution accidentally. Features specified
via `--features` and `--no-default-features` are spec'd as only applying to the
*current* package, not all packages.

Closes #3279

Auto merge of #3278 - alexcrichton:bump, r=alexcrichton

Bump to 0.15.0

Bump to 0.15.0

Auto merge of #3274 - alexcrichton:more-travis-again, r=alexcrichton

More travis again

Landing this again hopefully with the doc fix

Don't use travis-cargo to upload docs

Revert "Revert "Continuously publish Cargo builds""

This reverts commit b008422e92295c2e4de5b7de2284d963b55a6f8c.

Auto merge of #3259 - alexcrichton:more-ro, r=brson

Open crate files readonly first

This allows Cargo to work with read-only `CARGO_HOME` directories where the
cache was prepared ahead of time.

Closes #3256

Auto merge of #3270 - brson:cargo-docs, r=alexcrichton

Revert "Continuously publish Cargo builds"

This reverts commit 7799014dc7330445f62e2f925573ca15dd8e8e03.

Fixes https://github.com/rust-lang/cargo/issues/3269.

The reason doc.crates.io doesn't display correctly is that travis-cargo writes its own index.html that is just a meta-redirect to the api docs, overwriting cargo's own docs.

r? @alexcrichton

Revert "Continuously publish Cargo builds"

This reverts commit 7799014dc7330445f62e2f925573ca15dd8e8e03.

Mix feature flags into fingerprint/metadata shorthash

Since building dependencies results in different libraries
depending on the feature flags, I added the feature flags
into the short_hash. This solves an issue when multiple crates
share a target directory or multiple targets share a common
library with divergent feature flag choice.

- Only link binaries, build scripts, and top level deps
- Handle dylibs differently (no metadata filename / linking)
- Fingerprint based on link_dst rather than file_stem.

This (sadly) limits the effects of dep caching to things
which don't produce a hard-link. Currently, this is only dependent
library crates. Stil a big win.

Add tests for feature freshness

Auto merge of #3239 - alexcrichton:travis, r=brson

Continuously publish Cargo builds

This commit tweaks Cargo's automation to continuously publish builds on Travis
an AppVeyor. Once this is merged we can hopefully turn off all buildbot
automation related to Cargo and purely rely on Travis and AppVeyor for this
repository. All CI matrices are ported over to Travis and AppVeyor and a new
musl build of Cargo is even added just to test out adding that for a spin.

Currently Cargo will upload the final artifact for each target to a directory
keyed by the commit hash to a new bucket on S3, rust-lang-cargo-dev. Once we're
happy with the builds then we can change this to `rust-lang-cargo` or anything
else at that point.

Continuously publish Cargo builds

This commit tweaks Cargo's automation to continuously publish builds on Travis
an AppVeyor. Once this is merged we can hopefully turn off all buildbot
automation related to Cargo and purely rely on Travis and AppVeyor for this
repository. All CI matrices are ported over to Travis and AppVeyor and a new
musl build of Cargo is even added just to test out adding that for a spin.

Currently Cargo will upload the final artifact for each target to a directory
keyed by the commit hash to a new bucket on S3, rust-lang-cargo-dev. Once we're
happy with the builds then we can change this to `rust-lang-cargo` or anything
else at that point.

Auto merge of #3257 - nipunn1313:slash, r=alexcrichton

Add a [/] macro expansion for tests

Converts to backslash on windows. Currently, we're using [..],
so this will tighten up the tests.

Revert tests/read-manifest.rs

Open crate files readonly first

This allows Cargo to work with read-only `CARGO_HOME` directories where the
cache was prepared ahead of time.

Closes #3256

Replace instances of src[..]*.rs with [/]

Replace {sep} with [/] as well

Fix error in build-script

Add a [/] macro expansion for tests

Converts to backslash on windows. Currently, we're using [..],
so this will tighten up the tests.

Auto merge of #3255 - alexcrichton:bump, r=alexcrichton

Update makefile to 0.14.0

Update makefile to 0.14.0

Auto merge of #3254 - alexcrichton:unignore, r=alexcrichton

Un-ignore proc-macro tests

Auto merge of #3249 - matklad:workspace-profiles, r=alexcrichton

Use a single profile set per workspace

This aims to close #3206.

I have not figured out how to do this 100% backward compatibly, that's why I want to discuss this separately, although a related PR (#3221) is already in flight.

The problem is this: suppose that you have a workspace with two members, A and B and that A includes a profiles section and B does not. Now, mentally `cd` inside B and run `cargo build`. Today, Cargo will use a default profile. We want it to use a profile from A. So here the silent behavior switch will inevitably occur :( Looks like we can't detect this situation.

So this PR just switches the behavior to always use root profiles, and to print a warning if a non-root package specifies profiles. Feel free to reuse it in any form for #3221 if that's convenient!

Un-ignore proc-macro tests

Auto merge of #3253 - nipunn1313:exe, r=alexcrichton

Add an [EXE] hamcrest substitution for file extension

This should make the testing a bit more precise as to whether
we expect targets of the form foo.exe vs foo-abc123.exe. I
was also considering adding a [/] substitution for the fwd
slash vs backslash [..], but thought I would leave that to another PR
if we thought that was a good idea.

Backticks to tests/build-script.rs

Build scripts don't have .exe appended

Fix bench_dylib for win

Add an [EXE] hamcrest substitution for file extension

This should make the testing a bit more precise as to whether
we expect targets of the form foo.exe vs foo-abc123.exe. I
was also considering adding a [/] substitution for the fwd
slash vs backslash [..], but thought I would leave that to another PR
if we thought that was a good idea.

Auto merge of #3248 - alexcrichton:bad2, r=brson

Default urls in [replace] to crates.io

The intention was to do this, and it mistakenly didn't happen!

Closes #3235

Default urls in [replace] to crates.io

The intention was to do this, and it mistakenly didn't happen!

Closes #3235

Use a single profile set per workspace

Auto merge of #3243 - alexcrichton:target-feature, r=brson

Expose rustc cfg values to build scripts

This commit is Cargo's portion of the implementation of [RFC 1721] where it will
expose values printed by `rustc --print cfg` to build scripts.

[RFC 1721]: https://github.com/rust-lang/rfcs/blob/master/text/1721-crt-static.md

This will in turn be used to communicate features like `-C
target-feature=+crt-static` which can be used to compile objects for statically
linking against the msvcrt on MSVC.

Auto merge of #3220 - alexcrichton:bad, r=brson

Load [replace] sections from lock files

This commit fixes a bug in Cargo where path-based [replace] dependencies were
accidentally not loaded from lock files. This meant that even with a lock
file some compilations could accidentally become nondeterministic. The fix here
is to just look at all path dependencies, even those specified through [replace]

Closes #3216

Auto merge of #3175 - alexcrichton:bench-abort, r=brson

Ignore `panic` configuration for test/bench profiles

Both of these profiles link to libtest, so it's invalid to configure them with
`panic="abort"`. To prevent confusing errors just ignore the configuration for
now.

Closes #3166

Expose rustc cfg values to build scripts

This commit is Cargo's portion of the implementation of [RFC 1721] where it will
expose values printed by `rustc --print cfg` to build scripts.

[RFC 1721]: https://github.com/rust-lang/rfcs/blob/master/text/1721-crt-static.md

This will in turn be used to communicate features like `-C
target-feature=+crt-static` which can be used to compile objects for statically
linking against the msvcrt on MSVC.

Auto merge of #3246 - alexcrichton:mtimes-in-the-past, r=alexcrichton

Continuation of #2880

Just applied a patch to help fix the tests.

Auto merge of #3247 - alexcrichton:doc-plugin, r=alexcrichton

Continuation of #3114

Added a test